Skip to content

Fix create-component script to build successfully and documentation to align#4538

Merged
manishgarg1 merged 5 commits intomicrosoft:masterfrom
aditima:aditima/create-component
Apr 13, 2018
Merged

Fix create-component script to build successfully and documentation to align#4538
manishgarg1 merged 5 commits intomicrosoft:masterfrom
aditima:aditima/create-component

Conversation

@aditima
Copy link
Copy Markdown
Contributor

@aditima aditima commented Apr 12, 2018

Pull request checklist

  • Addresses an existing issue: Fixes #0000
  • Include a change request file using $ npm run change

Description of changes

  • Fix create-component script to build successfully without lint errors
  • NewComponent documentation to align to new output
  • Fix link in readme
  • Minor indentation fixes

Focus areas to test

(optional)

@aditima aditima requested a review from dzearing as a code owner April 12, 2018 23:24
@aditima aditima requested a review from manishgarg1 April 12, 2018 23:25
1. This is the recommended way of creating a new component.
3. Open a command prompt in the root directory of your project directory.
2. Run `npm run create-component -- --name ExcitingNewComponent`.
2. Open a command prompt in the root directory of your project directory.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

directory [](start = 63, length = 9)

Please remove this last "directory"

Copy link
Copy Markdown
Collaborator

@manishgarg1 manishgarg1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@manishgarg1 manishgarg1 merged commit 73c781f into microsoft:master Apr 13, 2018
@microsoft microsoft locked as resolved and limited conversation to collaborators Aug 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants